Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix!: datetime and collection id validation fixes as improve collection extent method #81

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

anayeaye
Copy link
Contributor

Release changes tested in dev

  • datetime validation timing fixed for more useful error messages
  • collection id validation fix allows adding items to previously published collections that are not kebab case
  • update collection extent step in dataset ingestion to use the newer pgstac v0.7 extent method that covers the full datetime range in the items partions

ividito and others added 4 commits July 18, 2023 14:43
- datetimes that do not exist are not checked for valid ranges
- collection ids that already exist are allowed to break rules
@anayeaye
Copy link
Contributor Author

@ividito I'm opening this to start the process but I have not addressed the release tagging yet

@anayeaye anayeaye changed the title ## Release datetime and collection id validation fixes as improve collection extent method Release datetime and collection id validation fixes as improve collection extent method Aug 17, 2023
@ividito ividito temporarily deployed to development September 19, 2023 21:07 — with GitHub Actions Inactive
@ividito ividito temporarily deployed to development September 22, 2023 17:31 — with GitHub Actions Inactive
@gadomski gadomski temporarily deployed to development October 20, 2023 17:52 — with GitHub Actions Inactive
@anayeaye anayeaye changed the title Release datetime and collection id validation fixes as improve collection extent method fix!: datetime and collection id validation fixes as improve collection extent method Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants